Skip to content
This repository has been archived by the owner on Oct 2, 2023. It is now read-only.

Add timeout for attr for dockerfile_image. #2230

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

charlesoconor
Copy link

@charlesoconor charlesoconor commented Mar 7, 2023

Some build take longer than 10 minutes since this doesn't use the docker cache so larger images can be slow.

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Currently the dockerfile_image will timeout after 10 minutes. Since there isn't a cache larger images builds can take longer than that especially on slow networks.

Issue Number: N/A

What is the new behavior?

This adds the option to set a higher timeout for the action.

Does this PR introduce a breaking change?

dockerfile_build

  • No

Some build take longer than 10 minutes since this doesn't use the docker
cache so larger images can be slow.
@charlesoconor charlesoconor force-pushed the add-timeout-option-for-docker-file branch from 46c9271 to 7653ba3 Compare March 7, 2023 03:23
@charlesoconor
Copy link
Author

Any idea when people might look at this? can close if it doesn't make sense, but I would like to know why.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant